Always include "__Secure-1PSIDTS" to avoid the "SNlM0e value not found" error. #254

SetoKaiba · 2023-12-28T05:06:16Z

I welcome the overall code refactoring and new features. However, please make sure to conduct QA. Before making a PR, please check the entire functionality by adding it to https://github.com/dsdanielpark/Bard-API/blob/main/func_test.ipynb. Due to various reasons, I couldn't implement test mocking code. I also welcome anyone who can provide test mocking and refactoring. But please keep the existing method names and examples. While I aim to accept PRs as quickly as possible, they may be rejected if QA is not correct.

BARD-API Pull Request Template

Please provide the necessary information as succinctly as possible. I appreciate it if you write it in an easy-to-read format as I don't have much time to dedicate to maintaining this package.

Description

Always include __Secure-1PSIDTS to avoid the "SNlM0e value not found" error.

If it uses __Secure-1PSID only, it will soon meet the SNlM0e value not found.
If it uses __Secure-1PSID and __Secure-1PSIDTS together, it won't.

Related Issue

#155

Motivation and Context

#155
I meet the SNlM0e value not found just after a few minutes. The __Secure-1PSID is not changed yet in the browser. But the Bard-API reports the problem. I found that using __Secure-1PSID and __Secure-1PSIDTS together will avoid this problem.

And also, I found some limitation of browser_cookie3.
I have cookies for .google.com.hk. But it filter the cookies by like sql. So the .google.com.hk and .google.com cookies are both in the results. The .google.com.hk cookie will override the .google.com cookie. It may cause the Bard API use the wrong values from .google.com.hk.
The problem comes from browser_cookie3. So this problem I can't find a way to fix it unless I delete the .google.com.hk cookies.

How Has This Been Tested?

I tested the sync version. It's working.

But the async version is not tested.
Because I found some problem even with the orignal one.

Traceback (most recent call last):
  File "E:\PycharmProjects\testGemini\test2.py", line 12, in <module>
    bard = BardAsync(token_from_browser=True, conversation_id='c_b3c979de2d5c535d')
  File "E:\PycharmProjects\testGemini\.venv\lib\site-packages\bardapi\core_async.py", line 80, in __init__
    self.SNlM0e = self._get_snim0e()
  File "E:\PycharmProjects\testGemini\.venv\lib\site-packages\bardapi\core_async.py", line 106, in _get_snim0e
    resp = self.client.get(
AttributeError: 'coroutine' object has no attribute 'get'
sys:1: RuntimeWarning: coroutine 'BardAsync._get_client' was never awaited

Thank you once again to all the contributors. Your efforts are greatly appreciated!

SetoKaiba · 2023-12-31T03:53:58Z

It seems that it doesn't need the whole cookies. But it always needs __Secure-1PSID, __Secure-1PSIDCC and __Secure-1PSIDTS.
The reference repo always includes __Secure_1PSIDTS as well.
https://github.com/acheong08/Bard
I'll modify the PR.

…d" error.

gitguardian · 2023-12-31T05:13:34Z

⚠️ GitGuardian has uncovered 3 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request

GitGuardian id	Secret	Commit	Filename
-	Google Bard API Key	`b1aa195`	func_test.ipynb	View secret
-	Generic High Entropy Secret	`b1aa195`	bardapi/core_async.py	View secret
-	Generic High Entropy Secret	`b1aa195`	bardapi/core_async.py	View secret

🛠 Guidelines to remediate hardcoded secrets

Understand the implications of revoking this secret by investigating where it is used in your code.
Replace and store your secrets safely. Learn here the best practices.
Revoke and rotate these secrets.
If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider

following these best practices for managing and storing secrets including API keys and other credentials
install secret detection on pre-commit to catch secret before it leaves your machine and ease remediation.

^{🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!}

dsdanielpark · 2024-01-01T13:45:26Z

https://github.com/dsdanielpark/Bard-API/blob/main/documents/README_DEV.md#multi-cookie-bard

Hello. Thank you very much for your wonderful contribution.

Regarding the cookie values, I have confirmed that in some regions, it is necessary to use two or three or more cookies. This has been addressed through Multi-cookies Bard. (Especially, users in China and Japan have reported this)

Generally, in most regions, accounts, and countries, a single cookie value is sufficient for a normal response, so I need to be a bit more cautious in accepting this PR. (The current code enforces the use of two cookie values)

Therefore, I will give this PR further consideration.

Thank you for your ideas and work.

SetoKaiba · 2024-01-02T09:21:26Z

https://github.com/dsdanielpark/Bard-API/blob/main/documents/README_DEV.md#multi-cookie-bard

Hello. Thank you very much for your wonderful contribution.

Regarding the cookie values, I have confirmed that in some regions, it is necessary to use two or three or more cookies. This has been addressed through Multi-cookies Bard. (Especially, users in China and Japan have reported this)

Generally, in most regions, accounts, and countries, a single cookie value is sufficient for a normal response, so I need to be a bit more cautious in accepting this PR. (The current code enforces the use of two cookie values)

Therefore, I will give this PR further consideration.

Thank you for your ideas and work.

OK. Then, for token from browser, I think that it should get the 3 cookies if it exists, not just __Secure-1PSID.

dsdanielpark · 2024-01-05T08:39:28Z

Thank you once again for your feedback. I will check to ensure that auto cookies can refresh three cookies. Also, please reconfirm if the issue you raised cannot be solved with multi-cookie bard as described here:

https://github.com/dsdanielpark/Bard-API/blob/main/documents/README_DEV.md#multi-cookie-bard

As shown in the example, assigning three cookies to a session object and passing it to bard should make it possible to use the Bard class with the intended functionality of using three cookie values.

Looking forward to your feedback. Thank you.

SetoKaiba mentioned this pull request Dec 28, 2023

Exception: SNlM0e value not found. Double-check __Secure-1PSID value or pass it as token='xxxxx'. #155

Closed

SetoKaiba changed the title ~~Use the whole cookies to avoid the "SNlM0e value not found" error.~~ Always include "__Secure-1PSIDTS" to avoid the "SNlM0e value not found" error. Dec 31, 2023

Always include "__Secure-1PSIDTS" to avoid the "SNlM0e value not foun…

b1aa195

…d" error.

SetoKaiba force-pushed the main branch from 1de9220 to b1aa195 Compare December 31, 2023 05:12

dsdanielpark closed this Jan 6, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Always include "__Secure-1PSIDTS" to avoid the "SNlM0e value not found" error. #254

Always include "__Secure-1PSIDTS" to avoid the "SNlM0e value not found" error. #254

SetoKaiba commented Dec 28, 2023 •

edited

Loading

SetoKaiba commented Dec 31, 2023 •

edited

Loading

gitguardian bot commented Dec 31, 2023

dsdanielpark commented Jan 1, 2024

SetoKaiba commented Jan 2, 2024 •

edited

Loading

dsdanielpark commented Jan 5, 2024

Always include "__Secure-1PSIDTS" to avoid the "SNlM0e value not found" error. #254

Always include "__Secure-1PSIDTS" to avoid the "SNlM0e value not found" error. #254

Conversation

SetoKaiba commented Dec 28, 2023 • edited Loading

BARD-API Pull Request Template

Description

Related Issue

Motivation and Context

How Has This Been Tested?

SetoKaiba commented Dec 31, 2023 • edited Loading

gitguardian bot commented Dec 31, 2023

⚠️ GitGuardian has uncovered 3 secrets following the scan of your pull request.

dsdanielpark commented Jan 1, 2024

SetoKaiba commented Jan 2, 2024 • edited Loading

dsdanielpark commented Jan 5, 2024

SetoKaiba commented Dec 28, 2023 •

edited

Loading

SetoKaiba commented Dec 31, 2023 •

edited

Loading

SetoKaiba commented Jan 2, 2024 •

edited

Loading